[Feat] Week4 필수 과제 #9
Conversation
| import kotlinx.serialization.Serializable | ||
|
|
||
| @Serializable | ||
| data class UserSignUpRequest( |
| fun Context.toast(message: String) { | ||
| Toast.makeText(this, message, Toast.LENGTH_SHORT).show() | ||
| } |
| password = password | ||
| ), | ||
| ) { body -> | ||
| editor.putString("loginToken", body!!.result.token) |
| fun postUserLogin( | ||
| context: Context, | ||
| body: UserLoginRequest, | ||
| callback: (ResponseUserTokenDto?) -> Unit |
There was a problem hiding this comment.
callback이 필요한 이유가 혹시 무엇일까요?? 중복된 로직을 제공한다면 불필요할 것 같아요!
| val error = response.message() | ||
| Log.e("error", error.toString()) |
There was a problem hiding this comment.
분기에 따른 에러 메시지가 보여지면 좋을 것 같아요! 지금은 로그인 실패 시 모두 fail_to_login 토스트 메시지가 뜨는 것 같아요!
| viewModel: MyViewModel = viewModel() | ||
| ) { | ||
| val context = LocalContext.current | ||
| val snackbarHostState = remember { SnackbarHostState() } | ||
| val sharedPreferences = context.getSharedPreferences("token", Context.MODE_PRIVATE) | ||
| val token = sharedPreferences.getString("loginToken", "").orEmpty() |
| if (response.isSuccessful) { | ||
| _userState.value = response.body() | ||
| _hobby.value = response.body()?.result?.hobby | ||
| Log.d("getUserHobby", response.body().toString()) |
There was a problem hiding this comment.
로그는 확인 후 지워줍시당 ! Timber를 활용해봐도 좋을 것 같아요 ~
| .addInterceptor(loggingInterceptor) | ||
| .build() | ||
|
|
||
| val retrofit: Retrofit by lazy { |
| .build() | ||
| } | ||
|
|
||
| inline fun <reified T> create(): T = retrofit.create(T::class.java) |
There was a problem hiding this comment.
처음 보는 코드여서 어떤 코드인지 찾아봤는데 재사용성이 더 좋아진다고하니 좋은 코드인 것 같아요~!
| @@ -184,16 +176,19 @@ fun LoginScreen( | |||
| WavveLoginButton( | |||
| onClick = { | |||
| viewModel.onLoginClick( | |||
There was a problem hiding this comment.
viewModel의 함수가 중첩돼서 가시성이 조금 떨어지는 것 같아요!
두개 함수를 조금 분리해서 파라미터로 넘기는 방식도 생각해봐주세요!
| LaunchedEffect(Unit) { | ||
| snackbarHostState.showSnackbar(context.getString(R.string.welcome)) | ||
| } | ||
| viewModel.getUserHobby(token = token) |
There was a problem hiding this comment.
직접 실행하는 방법도 좋은데 LaunchedEffect(Unit)을 사용해서 한 번만 불러오는 방법도 생각해주세요!
There was a problem hiding this comment.
이렇게 하면 리컴포지션시마다 계속 서버 통신 함수를 부르게 될 것 같네요!
| data class ResponseUserHobbyDto( | ||
| @SerialName("result") | ||
| val result: Result |
There was a problem hiding this comment.
형태가 계속 반복되니 baseResponse를 사용해봐도 좋을 듯 합니다.
| val dispatcher = LocalOnBackPressedDispatcherOwner.current!!.onBackPressedDispatcher | ||
| val sharedPreferences = context.getSharedPreferences("token", Context.MODE_PRIVATE) |
| LaunchedEffect(Unit) { | ||
| snackbarHostState.showSnackbar(context.getString(R.string.welcome)) | ||
| } | ||
| viewModel.getUserHobby(token = token) |
There was a problem hiding this comment.
이렇게 하면 리컴포지션시마다 계속 서버 통신 함수를 부르게 될 것 같네요!
Related issue 🛠
Work Description ✏️
회원가입
로그인
내 취미 조회
Screenshot 📸
Screen_recording_20241114_222915.mp4
Uncompleted Tasks 😅
To Reviewers 📢
피드백 뭐든지 환영합니당